Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FROST cfrg spec #119

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

FROST cfrg spec #119

wants to merge 1 commit into from

Conversation

nickfarrow
Copy link
Collaborator

@nickfarrow
Copy link
Collaborator Author

The latest draft-irtf-cfrg-frost-08 differs from v04 by having a per signer rho instead of a single group rho. See cfrg/draft-irtf-cfrg-frost#214. Thinking I'll match 04 first, then explore per signer rhos.

Current discrepancies:

  • My commitment hash participant index uses u8 (L862) they uses u16
  • Currently using the same binding coeff hash on the message

In Keygen, Jesse's implementation uses some compute_indexhash so indexhash = tagged_hash(pk || idx)
https://github.com/ElementsProject/secp256k1-zkp/blob/6c1cf4e0caac3a933b4f8cdbe14be05ea4c7c0d9/src/modules/frost/keygen_impl.h#L56

@nickfarrow
Copy link
Collaborator Author

nickfarrow commented Feb 21, 2023

Currently we use BTreeMaps for sorting nonces:

        let mut nonce_map: BTreeMap<_, _> =
            nonces.into_iter().map(|(i, nonce)| (i, nonce)).collect();

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant